-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: slider, range slider, checkbox, and content #439
Conversation
@mofojed I commented out each prop that I omitted along with the reason they're not included. I thought this would be easier to see which things were left intentionally but if it's preferred to not see them at all I can change it I also couldn't test the Content component properly. Wasn't sure how to pass in a child component into it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to fix imports, this does not install/run correctly and tests are not running as it is.
52298d7
to
92e0b5e
Compare
Closes #297, #298, #299, #440